Skip to content

MIDI APIs now use NotAllowedError instead of InvalidAccessError#42334

Merged
wbamberg merged 1 commit intomdn:mainfrom
nikelborm:fixed-deprecated-midi-invalid-access-error
Dec 19, 2025
Merged

MIDI APIs now use NotAllowedError instead of InvalidAccessError#42334
wbamberg merged 1 commit intomdn:mainfrom
nikelborm:fixed-deprecated-midi-invalid-access-error

Conversation

@nikelborm
Copy link
Contributor

@nikelborm nikelborm commented Dec 9, 2025

Motivation

Quoted from WebAudio/web-midi-api#278 PR description:

According to Web IDL spec on InvalidAccessError:

Deprecated. Use TypeError for invalid arguments, "NotSupportedError" DOMException for unsupported operations, and "NotAllowedError" DOMException for denied requests instead.

Previously, while trying to open MIDIPort, "InvalidAccessError" DOMException was thrown when...

the device is unavailable (e.g., is already in use by another process and cannot be opened, or is disconnected)

In this case, I believe NotAllowedError fits well, according to its Web IDL spec description:

The request is not allowed by the user agent or the platform in the current context, possibly because the user denied permission.

Security and permissions reasons are only a subset of the appropriate causes, according to this definition. An attempt to open the port can fail. Somebody's exclusive lock on the port, or the port being disconnected, is the current context driving the platform's decision not to allow the operation.

And in the context of forbidden .send calls:

If data is a System Exclusive message, and the MIDIAccess did not enable System Exclusive access, throw an InvalidAccessError exception.

NotAllowedError fits even better, because it fits an explicitly defined subset of permission errors.

Related issues and pull requests

The change in spec: WebAudio/web-midi-api@3795f22

This PR should not be confused with #41956, which talks about SecurityErrors instead

@nikelborm nikelborm requested a review from a team as a code owner December 9, 2025 23:10
@nikelborm nikelborm requested review from wbamberg and removed request for a team December 9, 2025 23:10
@github-actions github-actions bot added Content:WebAPI Web API docs size/xs [PR only] 0-5 LoC changed labels Dec 9, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 9, 2025

Preview URLs

Flaws (2)

URL: /en-US/docs/Web/API/MIDIOutput/send
Title: MIDIOutput: send() method
Flaw count: 1

  • broken_links:
    • /en-US/docs/Web/Security/Secure_Contexts is a redirect

URL: /en-US/docs/Web/API/MIDIPort/open
Title: MIDIPort: open() method
Flaw count: 1

  • broken_links:
    • /en-US/docs/Web/Security/Secure_Contexts is a redirect

@nikelborm nikelborm changed the title MIDI APIs now use NotAllowedError instead of InvalidAccessError MIDI APIs now use NotAllowedError instead of InvalidAccessError Dec 10, 2025
wbamberg
wbamberg approved these changes Dec 19, 2025
View reviewed changes
Copy link
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 thank you!

@wbamberg wbamberg merged commit ae1080c into mdn:main Dec 19, 2025
7 checks passed
nikelborm added a commit to nikelborm/firefox that referenced this pull request Dec 19, 2025
@nikelborm
Bug 2007071 - Align MIDI errors with the web-midi-api spec. Deprecate…
21a2174 
…d `SecurityError` and `InvalidAccessError` in favor of `NotAllowedError` r?gsvelto,bholley

= `InvalidAccessError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@3795f22](WebAudio/web-midi-api@3795f22)
According PR: [WebAudio/web-midi-api#278](WebAudio/web-midi-api#278)
MDN update PR: [mdn/content#42334](mdn/content#42334)

== Affected APIs

1. `.send` method of output MIDI ports
2. `.open` method of MIDI ports

= `SecurityError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@b7806b8](WebAudio/web-midi-api@b7806b8)
According PR: [WebAudio/web-midi-api#267](WebAudio/web-midi-api#267)
MDN update PR: [mdn/content#41956](mdn/content#41956)

== Affected APIs

1. `navigator.requestMIDIAccess` method
@nikelborm nikelborm mentioned this pull request Dec 19, 2025
Closed
nikelborm added a commit to nikelborm/firefox that referenced this pull request Dec 25, 2025
@nikelborm
Bug 2007071 - Align MIDI errors with the web-midi-api spec. Deprecate…
c5078c9 
…d `SecurityError` and `InvalidAccessError` in favor of `NotAllowedError` r?gsvelto

= `InvalidAccessError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@3795f22](WebAudio/web-midi-api@3795f22)
According PR: [WebAudio/web-midi-api#278](WebAudio/web-midi-api#278)
MDN update PR: [mdn/content#42334](mdn/content#42334)

== Affected APIs

1. `.send` method of output MIDI ports
2. `.open` method of MIDI ports

= `SecurityError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@b7806b8](WebAudio/web-midi-api@b7806b8)
According PR: [WebAudio/web-midi-api#267](WebAudio/web-midi-api#267)
MDN update PR: [mdn/content#41956](mdn/content#41956)

== Affected APIs

1. `navigator.requestMIDIAccess` method
nikelborm added a commit to nikelborm/firefox that referenced this pull request Dec 25, 2025
@nikelborm
Bug 2007071 - Align MIDI errors with the web-midi-api spec. Deprecate…
e0de00f 
…d `SecurityError` and `InvalidAccessError` in favor of `NotAllowedError` r?gsvelto

= `InvalidAccessError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@3795f22](WebAudio/web-midi-api@3795f22)
According PR: [WebAudio/web-midi-api#278](WebAudio/web-midi-api#278)
MDN update PR: [mdn/content#42334](mdn/content#42334)

== Affected APIs

1. `.send` method of output MIDI ports
2. `.open` method of MIDI ports

= `SecurityError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@b7806b8](WebAudio/web-midi-api@b7806b8)
According PR: [WebAudio/web-midi-api#267](WebAudio/web-midi-api#267)
MDN update PR: [mdn/content#41956](mdn/content#41956)

== Affected APIs

1. `navigator.requestMIDIAccess` method
nikelborm added a commit to nikelborm/firefox that referenced this pull request Dec 25, 2025
@nikelborm
Bug 2007071 - Align MIDI errors with the web-midi-api spec. Deprecate…
9defbe0 
…d `SecurityError` and `InvalidAccessError` in favor of `NotAllowedError` r=gsvelto

= `InvalidAccessError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@3795f22](WebAudio/web-midi-api@3795f22)
According PR: [WebAudio/web-midi-api#278](WebAudio/web-midi-api#278)
MDN update PR: [mdn/content#42334](mdn/content#42334)

== Affected APIs

1. `.send` method of output MIDI ports
2. `.open` method of MIDI ports

= `SecurityError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@b7806b8](WebAudio/web-midi-api@b7806b8)
According PR: [WebAudio/web-midi-api#267](WebAudio/web-midi-api#267)
MDN update PR: [mdn/content#41956](mdn/content#41956)

== Affected APIs

1. `navigator.requestMIDIAccess` method
nikelborm added a commit to nikelborm/firefox that referenced this pull request Dec 25, 2025
@nikelborm
Bug 2007071 - Align MIDI errors with the web-midi-api spec. Deprecate…
edd9946 
…d `SecurityError` and `InvalidAccessError` in favor of `NotAllowedError` r=gsvelto

= `InvalidAccessError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@3795f22](WebAudio/web-midi-api@3795f22)
According PR: [WebAudio/web-midi-api#278](WebAudio/web-midi-api#278)
MDN update PR: [mdn/content#42334](mdn/content#42334)

== Affected APIs

1. `.send` method of output MIDI ports
2. `.open` method of MIDI ports

= `SecurityError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@b7806b8](WebAudio/web-midi-api@b7806b8)
According PR: [WebAudio/web-midi-api#267](WebAudio/web-midi-api#267)
MDN update PR: [mdn/content#41956](mdn/content#41956)

== Affected APIs

1. `navigator.requestMIDIAccess` method

Differential Revision: https://phabricator.services.mozilla.com/D277533
nikelborm added a commit to nikelborm/firefox that referenced this pull request Dec 25, 2025
@nikelborm
Bug 2007071 - Align MIDI errors with the web-midi-api spec. Deprecate…
2e94450 
…d `SecurityError` and `InvalidAccessError` in favor of `NotAllowedError` r=gsvelto

= `InvalidAccessError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@3795f22](WebAudio/web-midi-api@3795f22)
According PR: [WebAudio/web-midi-api#278](WebAudio/web-midi-api#278)
MDN update PR: [mdn/content#42334](mdn/content#42334)

== Affected APIs

1. `.send` method of output MIDI ports
2. `.open` method of MIDI ports

= `SecurityError` replaced by `NotAllowedError`:

The change in spec: [WebAudio/web-midi-api@b7806b8](WebAudio/web-midi-api@b7806b8)
According PR: [WebAudio/web-midi-api#267](WebAudio/web-midi-api#267)
MDN update PR: [mdn/content#41956](mdn/content#41956)

== Affected APIs

1. `navigator.requestMIDIAccess` method

Differential Revision: https://phabricator.services.mozilla.com/D277533
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wbamberg wbamberg wbamberg approved these changes

Assignees

No one assigned

Labels

Content:WebAPI Web API docs size/xs [PR only] 0-5 LoC changed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nikelborm @wbamberg